Skip to content

Fix E4 model perspectives being converted to orphan local copies#3972

Merged
akurtakov merged 1 commit intoeclipse-platform:masterfrom
akurtakov:master
May 7, 2026
Merged

Fix E4 model perspectives being converted to orphan local copies#3972
akurtakov merged 1 commit intoeclipse-platform:masterfrom
akurtakov:master

Conversation

@akurtakov
Copy link
Copy Markdown
Member

E4 model perspectives are being made into local copies as it counted them as "orphan" with log like:

!ENTRY org.eclipse.ui 1 1 2026-05-06 12:00:28.236
!MESSAGE Perspective with name 'Some name' and id 'org.some.id' has been
made into a local copy

Perspectives with contributorURI in fragments are fully defined and not orphan thus add them directly without going through orphan and local copy.

@akurtakov
Copy link
Copy Markdown
Member Author

Problem can be seen with IWorkbenchPageTeset.testE4PerspectiveNotConvertedToLocalCopy and WorkbenchPage changes are reverted.

Copy link
Copy Markdown
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one typo and one question about the test cleanup. Otherwise it looks fine.

E4 model perspectives are being made into local copies as it counted
them as "orphan" with log like:
```
!ENTRY org.eclipse.ui 1 1 2026-05-06 12:00:28.236
!MESSAGE Perspective with name 'Some name' and id 'org.some.id' has been
made into a local copy
```

Perspectives with contributorURI in fragments are fully defined and not
orphan thus add them directly without going through orphan and local
copy.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Test Results

   852 files  ±0     852 suites  ±0   57m 12s ⏱️ + 5m 21s
 7 941 tests +1   7 698 ✅ +1  243 💤 ±0  0 ❌ ±0 
20 325 runs  +3  19 670 ✅ +3  655 💤 ±0  0 ❌ ±0 

Results for commit 44e4be7. ± Comparison against base commit 7f682c2.

♻️ This comment has been updated with latest results.

@vogella
Copy link
Copy Markdown
Contributor

vogella commented May 6, 2026

LGTM

@akurtakov akurtakov merged commit 4e91417 into eclipse-platform:master May 7, 2026
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants